Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create module tests #1303

Merged
merged 55 commits into from
Oct 10, 2023
Merged

Create module tests #1303

merged 55 commits into from
Oct 10, 2023

Conversation

annacv
Copy link
Contributor

@annacv annacv commented Sep 26, 2023

mgg709 and others added 9 commits September 13, 2023 12:43
* feat(xcontrols): added files for create skeleton

Co-authored-by: Ana Fernández Ostio <[email protected]>

* fix(xcontrols): rename package + add skeleton

Basic Skeleton added to some files that were empty

Co-Authored-By: Matías García Gimeno <[email protected]>
Co-Authored-By: Ger-28 <[email protected]>

---------

Co-authored-by: Ana Fernández Ostio <[email protected]>
Co-authored-by: Ana Fernández Ostio <[email protected]>
Co-authored-by: Ger-28 <[email protected]>
EMP-2056

---------

Co-authored-by: Matias Garcia <[email protected]>
Co-authored-by: Ana Fernández Ostio <[email protected]>
Co-authored-by: Matías García Gimeno <[email protected]>
Co-authored-by: Ger-28 <[email protected]>
EMP-2056

---------

Co-authored-by: Ana Fernández Ostio <[email protected]>
Co-authored-by: Ana Fernández Ostio <[email protected]>
Co-authored-by: Ana Fernández Ostio <[email protected]>
EMP-2064

---------

Co-authored-by: Ana Fernández Ostio <[email protected]>
Co-authored-by: Ana Fernández Ostio <[email protected]>
Co-authored-by: Ana Fernández Ostio <[email protected]>
EMP-2056

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Matias Garcia <[email protected]>
Co-authored-by: Ana Fernández Ostio <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Matías García Gimeno <[email protected]>
Co-authored-by: Guillermo Cacheda <[email protected]>
Co-authored-by: empathy/x <[email protected]>
Co-authored-by: acondal <[email protected]>
Co-authored-by: mariorey <[email protected]>
Co-authored-by: Gerardo Vázquez <[email protected]>
Co-authored-by: lauramargar <[email protected]>
Co-authored-by: Ger-28 <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@annacv annacv changed the base branch from main to feature/EMP-1944-create-config-module-xcontrols September 26, 2023 13:23
response: unknown
): (url: string, params: RequestInit) => Promise<Response> {
// eslint-disable-next-line @typescript-eslint/no-unused-vars
return _url => {
Copy link
Contributor Author

@annacv annacv Sep 26, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀 Signal not included here as it threw an error.

mgg709 and others added 17 commits September 27, 2023 11:35
EMP-2048

---------

Co-authored-by: Matías García Gimeno <[email protected]>
Co-authored-by: Ger-28 <[email protected]>
…te-endpoint-adapter

# Conflicts:
#	packages/x-types/package.json
#	pnpm-lock.yaml
@annacv annacv requested a review from CachedaCodes October 5, 2023 14:35
@annacv annacv marked this pull request as ready for review October 5, 2023 14:41
@annacv annacv requested a review from a team as a code owner October 5, 2023 14:41
@annacv annacv changed the base branch from feature/EMP-1944-create-config-module-xcontrols to main October 5, 2023 14:42
@annacv annacv requested a review from CachedaCodes October 9, 2023 15:13
@annacv annacv requested a review from CachedaCodes October 10, 2023 09:35
@CachedaCodes CachedaCodes merged commit 98acc0f into main Oct 10, 2023
3 checks passed
@CachedaCodes CachedaCodes deleted the feature/EMP-2182-Create-module-tests branch October 10, 2023 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants